Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New EPFL icons #716

Merged
merged 3 commits into from
Jun 18, 2024
Merged

New EPFL icons #716

merged 3 commits into from
Jun 18, 2024

Conversation

xentenza
Copy link
Collaborator

A bunch of 147 new EPFL icons

A bunch of 147 new EPFL icons
@xentenza
Copy link
Collaborator Author

@williambelle @alinekeller Je travaille sur cette PR (nouveaux icônes EPFL dans le Styleguide).

La nomenclature a changé. J'ai donc les deux problèmes suivants:

  1. certains icônes existent déjà sous un autre nom.
    Exemple: #icon-epfl-einstein vs #icon-epfl-science-einstein-invent
    Est-ce que je supprime les anciens fichiers ou les laisse, au risque de créer quelques petits effets de bord (soyons honnêtes, je ne suis pas sûr que beaucoup de gens utilisent encore ces icônes);

  2. Idem pour les anciens icônes ne figurant pas dans le nouveau pack: est-ce que je leur laisse le nom actuel, ou je les intègre dans la nouvelle nomenclature de Mediacom Communication Visuelle
    Exemple: transformer #icon-epfl-train en #icon-epfl-transport-rain (effets de bord identiques que ci-dessus)

Ou alors tant pis, certains icônes sont à double avec différents noms, et on continue avec la nouvelle nomenclature pour les icônes qui rejoindront le styleguide dans les prochaines années.

Merci d'avance pour vos idées!

@williambelle
Copy link
Member

Je pense qu'on peut supprimer et nettoyer les icônes.

L'idée serait de regrouper tous les "breaking changes" à un moment et ensuite faire une release en version 5.0.
Il faudrait choisir une fenêtre pour intégrer ces modifications sans bloquer les développements pour WordPress (ou autres).

- added new icons by Diglu
- deleted duplicated ones
@xentenza
Copy link
Collaborator Author

xentenza commented May 8, 2024

Hello @williambelle,
Ça devrait être bon. Le principal effet de bord que je vois c'est
#icon-epfl-graph-search
utilisé dans search.epfl.ch et qui a été renommé
#icon-epfl-science-graph-search

@xentenza xentenza removed the question label May 8, 2024
@xentenza xentenza marked this pull request as ready for review May 15, 2024 14:16
@alinekeller
Copy link
Contributor

Je suis du même avis que William, ça me semble plus propre d'éviter les doublons. Les effets de bord potentiels me semblent rapides à corriger (et pas gravissimes, puisqu'en théorie un pictogramme ne devrait pas être utilisé seul pour véhiculer une information).

Copy link

Unit Test Results

    1 files      1 suites   0s ⏱️
273 tests 249 ✔️ 0 💤   0  24 🔥
273 runs  225 ✔️ 0 💤 24  24 🔥

For more details on these errors, see this check.

Results for commit 52c3e98.

Copy link

🔎 Download the Backstop report for this pull request (link valid for 90 days):

@williambelle
Copy link
Member

Ça devrait être bon. Le principal effet de bord que je vois c'est
#icon-epfl-graph-search
utilisé dans search.epfl.ch et qui a été renommé
#icon-epfl-science-graph-search

Ça tombe bien, c'est moi qui m'en occupe.

@williambelle williambelle merged commit 680921b into dev Jun 18, 2024
4 checks passed
@williambelle williambelle deleted the styleguide/new-epfl-icons branch June 18, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants